Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add telemetry #4740

Merged
merged 1 commit into from
Nov 22, 2024
Merged

add telemetry #4740

merged 1 commit into from
Nov 22, 2024

Conversation

msarahan
Copy link
Contributor

@msarahan msarahan commented Oct 29, 2024

Enables telemetry during cugraph's build process. This parses github job metadata to obtain timing information. It should have very little impact on overall build time, and should not interfere with any build tools.

This implements emitting OpenTelemetry traces and spans, as described in https://github.com/rapidsai/build-infra/issues/139

@msarahan msarahan added improvement Improvement / enhancement to an existing function DO NOT MERGE Hold off on merging; see PR for details non-breaking Non-breaking change ci Needs build-infra Requires input from the build infrastructure team labels Oct 29, 2024
@github-actions github-actions bot removed the ci label Oct 29, 2024
@msarahan msarahan removed the DO NOT MERGE Hold off on merging; see PR for details label Nov 4, 2024
@msarahan msarahan marked this pull request as ready for review November 4, 2024 16:48
@msarahan msarahan requested a review from a team as a code owner November 4, 2024 16:48
@msarahan msarahan requested a review from jameslamb November 4, 2024 16:48
@msarahan msarahan changed the title add telemetry (testing) add telemetry Nov 15, 2024
@msarahan
Copy link
Contributor Author

/merge

@rapids-bot rapids-bot bot merged commit 460d8e4 into rapidsai:branch-24.12 Nov 22, 2024
73 checks passed
@msarahan msarahan deleted the add-telemetry branch November 26, 2024 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement Improvement / enhancement to an existing function Needs build-infra Requires input from the build infrastructure team non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants